Skip to content

Conversation

@art-w
Copy link
Collaborator

@art-w art-w commented Nov 10, 2025

Follow-up on the instantiation of parameterised libraries #12561 to support inline_tests (marking this PR as a draft since only the last commit is new).

To run the inline tests of a parameterised library, the user should specify with (arguments ...) which implementation of the parameters to use, as otherwise the test can't be ran.

Fix #12110

@Alizter
Copy link
Collaborator

Alizter commented Nov 10, 2025

Is ppx_expect building in oxcaml now?

@art-w
Copy link
Collaborator Author

art-w commented Nov 10, 2025

ppx_inline_test works yes, but I think the nix flake is using a more edgy version of OxCaml than the one published in their opam-repo... I'm looking into it.

Copy link
Member

@shonfeder shonfeder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! I have just a few questions and 2 small suggestions

@art-w art-w force-pushed the parameterised-test branch 2 times, most recently from f309b12 to c0466c9 Compare December 2, 2025 08:56
@art-w art-w force-pushed the parameterised-test branch from 5d6125e to e0e2764 Compare December 2, 2025 09:34
@rgrinberg rgrinberg added package management oxcaml Related to the support to OxCaml functionnalities and removed package management labels Dec 2, 2025
@art-w art-w force-pushed the parameterised-test branch 2 times, most recently from 1549c9c to 2d5c192 Compare December 5, 2025 10:56
@art-w art-w force-pushed the parameterised-test branch 2 times, most recently from 582309b to 5485df8 Compare December 8, 2025 11:13
@art-w art-w force-pushed the parameterised-test branch from 5485df8 to 98ce06a Compare December 8, 2025 11:19
@Alizter Alizter requested review from Alizter and rgrinberg December 8, 2025 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

oxcaml Related to the support to OxCaml functionnalities

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[OxCaml] Parameterized libraries: support for applying library arguments in inline tests

4 participants